Skip to content

Ignore the FLATC_EXECUTABLE env var when empty#3813

Merged
mergennachin merged 1 commit intopytorch:release/0.2from
dbort:flatc-empty-path
Jun 3, 2024
Merged

Ignore the FLATC_EXECUTABLE env var when empty#3813
mergennachin merged 1 commit intopytorch:release/0.2from
dbort:flatc-empty-path

Conversation

@dbort
Copy link
Copy Markdown
Contributor

@dbort dbort commented Jun 3, 2024

We've seen some situations fail because flatc_path is an empty string. This seems like a likely culprit, but doesn't hurt either way.

We've seen some situations fail because flatc_path is an empty string.
This seems like a likely culprit, but doesn't hurt either way.
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Jun 3, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/3813

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit fee0ed3 with merge base d3326a2 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 3, 2024
@dbort dbort mentioned this pull request Jun 3, 2024
@dbort dbort requested review from Gasoonjia and kirklandsign June 3, 2024 18:46
@dbort
Copy link
Copy Markdown
Contributor Author

dbort commented Jun 3, 2024

Cherry-pick for 0.2.1

@dbort dbort requested review from mcr229 and removed request for kirklandsign June 3, 2024 18:48
@mergennachin mergennachin merged commit 6c74717 into pytorch:release/0.2 Jun 3, 2024
@dbort dbort deleted the flatc-empty-path branch June 3, 2024 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants